-
Notifications
You must be signed in to change notification settings - Fork 27
feat: add keda autoscaler for container and application service (only… #2265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Qovery can create a Preview Environment for this PR.
This comment has been generated from Qovery AI 🤖.
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## staging #2265 +/- ##
===========================================
- Coverage 47.73% 47.42% -0.32%
===========================================
Files 1254 1265 +11
Lines 22717 23317 +600
Branches 6635 6869 +234
===========================================
+ Hits 10844 11057 +213
- Misses 9808 10131 +323
- Partials 2065 2129 +64
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
041a81e to
aed396c
Compare
bf0d6ba to
9d0dfd3
Compare
RemiBonnet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Pierre 🙏
I added a few comments, but I think it would be better if I rewrite this part
The logic feels too complex for what are just small settings, and it currently affects some sensitive parts of the codebase. I think this logic should be isolated, as it is now, it’s hard to maintain and risky. Let’s talk about it tomorrow!
libs/shared/ui/src/lib/components/inputs/input-text-area/input-text-area.tsx
Outdated
Show resolved
Hide resolved
libs/shared/util-services/src/lib/build-edit-service-payload.ts
Outdated
Show resolved
Hide resolved
| min_running_instances: number | ||
| max_running_instances: number | ||
|
|
||
| // Autoscaling mode selection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we have an specific interface for keda and extends this one? (you can probably extends from the api doc)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pggb25 Indeed that would be nice 🙏
...ib/application-settings/ui/application-settings-resources/application-settings-resources.tsx
Show resolved
Hide resolved
...lication/src/lib/feature/page-settings-resources-feature/page-settings-resources-feature.tsx
Outdated
Show resolved
Hide resolved
...lication/src/lib/feature/page-settings-resources-feature/page-settings-resources-feature.tsx
Outdated
Show resolved
Hide resolved
...lication/src/lib/feature/page-settings-resources-feature/page-settings-resources-feature.tsx
Outdated
Show resolved
Hide resolved
...lication/src/lib/feature/page-settings-resources-feature/page-settings-resources-feature.tsx
Outdated
Show resolved
Hide resolved
...lication/src/lib/feature/page-settings-resources-feature/page-settings-resources-feature.tsx
Outdated
Show resolved
Hide resolved
7ec2455 to
7a94741
Compare
RemiBonnet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Pierre!
I added few comments and if you can all new files inside console-shared should be in your folder keda same for new files inside the util-services 🙏
libs/domains/services/feature/src/lib/keda/scaled-object-status/scaled-object-status.tsx
Outdated
Show resolved
Hide resolved
| @@ -0,0 +1,88 @@ | |||
| import type { ScaledObjectStatusDto } from 'qovery-ws-typescript-axios/dist/api' | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| import type { ScaledObjectStatusDto } from 'qovery-ws-typescript-axios/dist/api' | |
| import type { ScaledObjectStatusDto } from 'qovery-ws-typescript-axios' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pggb25 This comment is still relevant, could you please apply @RemiBonnet's suggestion?
.../application/src/lib/feature/page-settings-general-feature/page-settings-general-feature.tsx
Show resolved
Hide resolved
...rc/lib/feature/page-application-create-feature/step-summary-feature/step-summary-feature.tsx
Show resolved
Hide resolved
.../lib/application-settings/ui/application-settings-resources/components/hpa-metric-fields.tsx
Outdated
Show resolved
Hide resolved
...ib/application-settings/ui/application-settings-resources/application-settings-resources.tsx
Show resolved
Hide resolved
...ib/application-settings/ui/application-settings-resources/application-settings-resources.tsx
Show resolved
Hide resolved
libs/shared/util-services/src/lib/keda/autoscaling-payload/autoscaling-payload.ts
Outdated
Show resolved
Hide resolved
libs/shared/util-services/src/lib/keda/autoscaling-payload/autoscaling-payload.ts
Outdated
Show resolved
Hide resolved
c57c64a to
900321c
Compare
libs/domains/services/feature/src/lib/keda/components/keda-settings.tsx
Outdated
Show resolved
Hide resolved
900321c to
5dbde4f
Compare
5dbde4f to
ddbdaa0
Compare
| @@ -0,0 +1,88 @@ | |||
| import type { ScaledObjectStatusDto } from 'qovery-ws-typescript-axios/dist/api' | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pggb25 This comment is still relevant, could you please apply @RemiBonnet's suggestion?
libs/domains/services/feature/src/lib/keda/components/instances-range-inputs.tsx
Outdated
Show resolved
Hide resolved
...ages/cluster/src/lib/feature/page-settings-general-feature/page-settings-general-feature.tsx
Outdated
Show resolved
Hide resolved
...ib/application-settings/ui/application-settings-resources/application-settings-resources.tsx
Outdated
Show resolved
Hide resolved
| min_running_instances: number | ||
| max_running_instances: number | ||
|
|
||
| // Autoscaling mode selection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pggb25 Indeed that would be nice 🙏
|
🎉 This PR is included in version 1.272.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
… qovery admin)
Summary
This PR allow to configure KEDA autoscaling for container and application service
Issue:
Screenshots / Recordings
Testing
yarn testoryarn test -u(if you need to regenerate snapshots)yarn formatyarn lintPR Checklist
.cursor/rules)feat(service): add new Terraform service) - required for semantic-release